Skip to content

Conversation

@jcpowermac
Copy link

@jcpowermac jcpowermac commented Oct 27, 2025

What this PR does / why we need it:

Installing OpenShift on vSphere, in our dev environments, can be difficult for a first timer.
This slash command (with skills, scripts to reduce token usage) flows through the usual
steps that someone needs to perform to install vSphere in our environments

Special notes for your reviewer:

Checklist:

  • Subject and description added to both, commit and PR.
  • Relevant issues have been referenced.
  • This change includes docs.

@openshift-ci
Copy link

openshift-ci bot commented Oct 27, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 27, 2025
@enxebre
Copy link
Contributor

enxebre commented Oct 28, 2025

thanks! is there anything specific about destructive / write operations for the CLIs used that needs to be called out explicitly in the spec for the model?
/approve

@openshift-ci
Copy link

openshift-ci bot commented Oct 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: enxebre, jcpowermac

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 28, 2025
For a new developer this can be a very difficult
experience. This is my first attempt
with Claude to create a command, it flows
through the usual steps that someone needs to perform
to install on vSphere in our environments.
@jcpowermac
Copy link
Author

thanks! is there anything specific about destructive / write operations for the CLIs used that needs to be called out explicitly in the spec for the model? /approve

It will download the ova, and then upload if there is not a template already available. I can't think of anything destructive

Looking through what Claude wrote yesterday, it missed alot, even after we went through testing it.

@jcpowermac jcpowermac marked this pull request as ready for review October 29, 2025 11:57
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 29, 2025
@openshift-ci openshift-ci bot requested review from dgoodwin and stbenjam October 29, 2025 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants